Bug 1614805 Comment 18 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Aaron, the failures in [comment 5](https://bugzilla.mozilla.org/show_bug.cgi?id=1614805#c5) are indeed due to e regex but are very rare and what I did is to add a try catch to be able to see the string that is failing. However, the zero-values aren't caused by that as they are permanent (they happen also when the matching error doesn't). So this issue has 2 culprits: silent failure because of the re.match and the zero-values whose occurrence I explained in [comment 15](https://bugzilla.mozilla.org/show_bug.cgi?id=1614805#c16).

As we can't really see what event name is causing the silent fail, my proposal is to land a patch adding a `try/except` to the `re.match` as the intermittent occurrence is fairly low.

I wouldn't wait one more week, as this patch is fairly easy to review, and it could buy us time to advance with the investigations about zero-values.
Thanks for your answer, Aaron.
Aaron, the failures in [comment 5](https://bugzilla.mozilla.org/show_bug.cgi?id=1614805#c5) are indeed due to e regex but are very rare and what I did is to add a try catch to be able to see the string that is failing. However, the zero-values aren't caused by that as they are permanent (they happen also when the matching error doesn't). So this issue has 2 culprits: silent failure because of the re.match and the zero-values whose occurrence I explained in [comment 15](https://bugzilla.mozilla.org/show_bug.cgi?id=1614805#c16). With the mention that the zero-values happen since ever while the match failure is relatively recent.

As we can't really see what event name is causing the silent fail, my proposal is to land a patch adding a `try/except` to the `re.match` as the intermittent occurrence is fairly low.

I wouldn't wait one more week, as this patch is fairly easy to review, and it could buy us time to advance with the investigations about zero-values.
Thanks for your answer, Aaron.

Back to Bug 1614805 Comment 18